Skip to content

Add optional timestamps enabled by env CUOPT_EXTRA_TIMESTAMPS#392

Merged
rapids-bot[bot] merged 4 commits intoNVIDIA:branch-25.10from
tmckayus:timestamps
Oct 3, 2025
Merged

Add optional timestamps enabled by env CUOPT_EXTRA_TIMESTAMPS#392
rapids-bot[bot] merged 4 commits intoNVIDIA:branch-25.10from
tmckayus:timestamps

Conversation

@tmckayus
Copy link
Contributor

This change adds optional internal timestamps which can be enabled with env var CUOPT_EXTRA_TIMESTAMPS=True.

The timestamps are to allow benchmarking scripts to key off of the events in order to calculate problem setup and teardown time.

@tmckayus tmckayus requested review from a team as code owners September 15, 2025 15:02
@copy-pr-bot
Copy link

copy-pr-bot bot commented Sep 15, 2025

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@tmckayus tmckayus added feature request New feature or request non-breaking Introduces a non-breaking change labels Sep 15, 2025
@tmckayus
Copy link
Contributor Author

/ok to test b9bfccf

@anandhkb anandhkb added this to the 25.10 milestone Sep 15, 2025
@tmckayus tmckayus requested a review from a team as a code owner September 16, 2025 19:28
@tmckayus
Copy link
Contributor Author

/ok to test 99c5ee9

Copy link
Member

@KyleFromNVIDIA KyleFromNVIDIA left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved trivial CMake changes

@rgsl888prabhu
Copy link
Collaborator

@rg20 @chris-maes May I get your review on this PR

Copy link
Contributor

@aliceb-nv aliceb-nv left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approving C++

@rgsl888prabhu
Copy link
Collaborator

/ok to test 9de0f56

@rgsl888prabhu
Copy link
Collaborator

/ok to test 92fa733

@aliceb-nv
Copy link
Contributor

/merge

@rapids-bot rapids-bot bot merged commit f4d6791 into NVIDIA:branch-25.10 Oct 3, 2025
172 of 174 checks passed
@tmckayus tmckayus deleted the timestamps branch February 5, 2026 14:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature request New feature or request non-breaking Introduces a non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants